Hello @Shalujha0907

Thank you for the PR!

I see this PR changes the expectation for CHARACTER from KRKT to XRKT, but isn't the correct value XRKTR?

Hello @Shalujha0907

Thank you for the PR!

I see this PR changes the expectation for CHARACTER from KRKT to XRKT, but isn't the correct value XRKTR?

Hello @garydgregory

Thanks for the review and for calling this out.

XRKTR is the full phonetic sequence, but Metaphone in Commons Codec uses a default maxCodeLen of 4, so the returned value is truncated to XRKT.
That’s why this test expects XRKT (and not XRKTR) unless setMaxCodeLen(...) is increased.

Please let me know if you think we should also add a separate assertion with a higher maxCodeLen to make that behavior explicit.

Hi @Shalujha0907
Ah, yes, you're right 👍
Would you please mind adding a new test for this specific 5 char use case?
Ty!

5 cha

Hi @Shalujha0907 Ah, yes, you're right 👍 Would you please mind adding a new test for this specific 5 char use case? Ty!

Hi @garydgregory
Pushed the latest change.

Thank you!

Jira ticket is https://issues.apache.org/jira/browse/CODEC-249